Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change topology test to listen only on localhost #22353

Merged
merged 1 commit into from
Jun 14, 2017
Merged

Conversation

amitmurthy
Copy link
Contributor

This will fix #21933 (comment) if Travis infrastructure is running some sort of a port scanner.

@amitmurthy
Copy link
Contributor Author

The error messages highlighted in #21933 (comment) , which used to appear in both Linux and Windows logs have been fixed by this PR.

However, there is still a different spurious error log in the 64-bit linux run (reproduced below).

EOFError()CapturedException(EOFError()CapturedException(EOFError(), EOFError(), 
Any[(Any[(unsafe_read(::Base.AbstractIOBuffer{Array{UInt8,1}}, ::Ptr{UInt8}, ::UInt64) at iobuffer.jl:105, 1), (unsafe_read(::TCPSocket, ::Ptr{UInt8}, ::UInt64) at stream.jl:752, 1), 
(unsafe_read(::TCPSocket, ::Base.RefValue{NTuple{4,Int64}}, ::Int64) at io.jl:362, 1), (read at 
io.jl:364 [inlined], 1), (deserialize_hdr_raw at messages.jl:170 [inlined], 1), \(message_handler_loop(::TCPSocket, ::TCPSocket, ::Bool) at process_messages.jl:157, 1), 
(process_tcp_streams(::TCPSocket, ::TCPSocket, ::Bool) at process_messages.jl:118, 1), 
(EOFError()CapturedException((::Base.Distributed.##99#100{TCPSocket,TCPSocket,Bool})
()EOFError(), Any[(

I'll go ahead and merge this PR in a day as it fixes one known issue.

@martinholters
Copy link
Member

BTW, should these make CI fail maybe? Otherwise, they can easily go unnoticed.

@amitmurthy
Copy link
Contributor Author

No, they are not errors. Any spurious connection attempt to a worker will result in a Unknown remote, closing connection. error message being displayed. I don't know why/who in the Travis/AV setup is attempting a connection.

I do not see them on my local OSX setup, nor did I see it during a test I ran on a Linux system.

@amitmurthy amitmurthy merged commit 17d5c55 into master Jun 14, 2017
@amitmurthy amitmurthy deleted the amitm/spurious branch June 14, 2017 05:14
@tkelman
Copy link
Contributor

tkelman commented Jun 18, 2017

This is still happening - https://travis-ci.org/JuliaLang/julia/jobs/244155807. Looks pretty alarming in the logs, and I've seen it locally too.

@StefanKarpinski
Copy link
Member

I think this is the same issue: https://travis-ci.org/JuliaLang/julia/jobs/244336112

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants